Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for continuous integration via travis #5

Merged
merged 13 commits into from
Apr 17, 2017
Merged

Conversation

jchodera
Copy link
Member

@jchodera jchodera commented Apr 15, 2017

This adds continuous integration support via travis.

@jchodera
Copy link
Member Author

I think travis is working now!

There are some failures due to python 3.x issues that are addressed separately in #3

@jchodera
Copy link
Member Author

jchodera commented Apr 16, 2017

To save time in future travis runs, we can cache the equilibrium data using this scheme.

We'd probably want to make it possible to override DATA_PATH by an environment variable $BENCHMARK_DATA_PATH so that we can cache the data in $HOME/data instead.

@jchodera
Copy link
Member Author

This is going to be super useful in debugging #3, so I'd suggest merging this PR first and then merging the latest changes into #3 when you pick things up again.

@maxentile
Copy link
Member

Ahh, thanks for putting this together! Just looked over all the changes, and they look good to me. Will make a couple minor edits to Yank-specific devtools files, then merge. I'm a bit puzzled by some of the Travis failures (e.g. No module named 'numpy'), but will look into this later...

To save time in future travis runs, we can cache the equilibrium data using this scheme. We'd probably want to make it possible to override DATA_PATH by an environment variable $BENCHMARK_DATA_PATH so that we can cache the data in $HOME/data instead.

Good idea! Will add an Issue to remind me to do this...

@maxentile maxentile merged commit 024dcd3 into master Apr 17, 2017
@maxentile maxentile deleted the travis branch April 17, 2017 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants